docs: add B20 concept docs - #1773
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
🟡 Heimdall Review Status
|
There was a problem hiding this comment.
Two issues, both from SEIZE_RECEIVER_POLICY (the 6th policy scope) being omitted. Inline with comments left in #1766.
Separately, on docs.json: there is no redirect for the exact old path /base-chain/specs/upgrades/beryl/b20 (the deleted single-page spec lived there). Redirects were added for /base-chain/b20, /base-chain/b20/*, and …/beryl/b20/{concepts,implementation,reference}/*, but not the bare …/beryl/b20, so that URL — and old deep links to its anchors (#multiplier, #policy-registry, #pause, #announcements), will 404 after merge.
roethke
left a comment
roethke
left a comment
There was a problem hiding this comment.
SEIZE_RECEIVER_POLICY fixes look good, thanks. One issue with the new redirect, inline.
| "destination": "/agents/:slug*" | ||
| }, | ||
| { | ||
| "source": "/base-chain/specs/upgrades/beryl/b20", |
There was a problem hiding this comment.
b20.mdx still exists at this path, so the page isn't removed. A redirect whose source is a live page makes that page unreachable. Either delete b20.mdx if it's being retired, or drop this redirect and keep the page in the nav.
This also conflicts with #1812, which keeps and edits b20.mdx and cross-links it. Worth aligning both PRs on whether the conceptual page stays or is retired.
3 participants
What changed? Why?
Added b20 concepts